gtkplacesview: allow to cancel connection to server

We were not allowing to cancel the operation at all, and at
most the operation was cancelled only when clicked connect again.

Also due to gvfs bug 753735 we actually weren't cancelling
at all, and therefore creating multiple dialogs.

places-view: show a more verbose entry hint

As pointed out by Allan, this is not very helpful. A
call to action label fits better in this case.

flowbox: Sink the floating reference of model-based children

We don't want to leak references if the widget created to represent the
item in the model does not have a floating reference — which is usually
what happens in bindings, as they automatically sink references when
creating new instances.

See commit 6e03e7e8 for the similar change in GtkListBox.

MSVC Introspection Builds: Remove GCC Requirement

Sorry, the last commit added a generated file instead of the
template.

G-I has been updated to not require a Windows GCC installation
anymore to generate the .gir files, so update the NMake Makefiles
that are used for this purpose.

As a result, it is no longer necessary to define time_t for the .gir
generation as we are on the same compiler throughout the process.

Revert "MSVC Introspection Builds: Remove GCC Requirement"

This reverts commit aea7809f55ac520b9d2d5d9c1eea1345fabbd7a7.

MSVC Introspection Builds: Remove GCC Requirement

G-I has been updated to not require a Windows GCC installation
anymore to generate the .gir files, so update the NMake Makefiles
that are used for this purpose.

As a result, it is no longer necessary to define time_t for the .gir
generation as we are on the same compiler throughout the process.

gtk-demo: Use top/bottom-margin for text views

This lets us replace the hack of inserting tags in the first
and last paragraph to set a above/below-paragraph space.

text view: add padding and margin

Add API to set top and bottom margins, and add the CSS padding
from the theme to the programmatically set margins.

https://bugzilla.gnome.org/show_bug.cgi?id=406159

Updated Galician translations

Add a test for gtk_flow_box_bind_model

Add gtk_flow_box_bind_model

Add a test for g_list_box_bind_model

list box: Shorten some static function names

Bump Pango requirement to 1.37.3

GTK+ now uses pango_attr_foreground_alpha_new, pango_attr_background_alpha_new,
PANGO_ATTR_FOREGROUND_ALPHA, PANGO_ATTR_BACKGROUND_ALPHA,
pango_renderer_set_alpha, pango_renderer_get_alpha, which were all added
after 1.37.2.

Updated Portuguese translation

popover: Don't restore stale default widget

Only restore the saved default widget if the default hasn't been
changed while the popover was visible.

Add a note to gdk_screen_get_width/height_mm

The values returned by these functions are not very well defined,
recommend to use the monitor dimensions instead.

wayland: calculate screen physical size

A simple calculation is done so far (assuming monitor areas never overlap)
so gdk_screen_get_width/height_mm return meaningful values.

https://bugzilla.gnome.org/show_bug.cgi?id=753621

popover: Optimize the transition

There's no need to queue a resize here.

https://bugzilla.gnome.org/show_bug.cgi?id=753568

window: Set name on idle

Use gdk_threads_add_idle, and set a name on the source to aid
debugging, as we do everywhere else.

searc engine model: Set name on idle

Use gdk_threads_add_idle, and set a name on the source to aid
debugging, as we do everywhere else.

Update release notes a bit

Fix a thinko in GtkLabel

Support alpha in text cell renderer colors

This is easily possible, now that Pango supports alpha.

Updated Slovak translation

GtkLabel: Add alpha support for buildable attributes

Make it possible to set foreground or background alpha in the
builder ui support for Pango attributes.

GtkTextDisplay: Use pango renderer alpha support

We don't need to store our own copy of the colors anymore,
now that PangoRenderer can do alpha.

Updates

Revert "wayland: require WAYLAND_DISPLAY be set to open wayland display"

This reverts commit 35489f5d31b20a390b45803d760f965391d0b57f.

It snuck in when i was trying to push

commit f4d2022d46e16e1fdec8ca9b107a9870155832bf

wayland: change wl_log level to G_LOG_LEVEL_DEBUG

wl_log() currently logs using G_LOG_LEVEL_ERROR
(which is fatal). The wayland client library doesn't
expect this behavior. It uses wl_log to log recoverable
errors.

This commit changes the log level to G_LOG_LEVEL_DEBUG
instead.

https://bugzilla.gnome.org/show_bug.cgi?id=753635

wayland: require WAYLAND_DISPLAY be set to open wayland display

The wayland client libraries now require WAYLAND_DISPLAY be set
to use them.

See:

http://cgit.freedesktop.org/wayland/wayland/commit/?id=fb7e13021730d0a5516ecbd3712ea4235e05d24d

This commit makes _gdk_wayland_display_open bail early if
WAYLAND_DISPLAY is unset, just as it does for XDG_RUNTIME_DIR.

GDK W32: Add missing commas

Revert "Revert "Revert "gtkwindow: Reset maximize / fullscreen_initially on state changes"""

This reverts commit 8001b2c1b3b9b5535d20ec942c497ae0f0a4adc7.

Revert "Revert "Revert "window: fix initial shadow width for maximized/fullscreen windows"""

This reverts commit 968ac90e89f444e180d883a28bfd260b118f451d.

Revert "Revert "gtkwindow: Reset maximize / fullscreen_initially on state changes""

This reverts commit de2ea1ebaabd7550ca6f2a02bfbaa04ac92181fd.

No reason in the commit message was given for the revert, and we need
this for proper behavior on map.

Revert "Revert "window: fix initial shadow width for maximized/fullscreen windows""

This reverts commit 8b695cc7279270eabf7d2541cfd592608d0add38.

No reason in the commit message was given for the revert, and we need
this for proper behavior on map.

wayland: Implement touchpad gesture events

On wayland, the gestures protocol defines a wl_pointer_gestures global
object, that will match in number with wl_seats, swipe and pinch
interfaces can be obtained from it, which events are translated into
GdkEventTouchpadSwipe/Pinch events.

wayland: Add touchpad gestures protocol

gtkgesture: Add note to gtk_gesture_get_bounding_box()

This is the only function that's will be semantically confusing
with touchpad gesture events, explain what's to be expected there.

gtkgesture: Add touchpad gesture event bit to the controller evmask

gtkgesturedrag: Handle touchpad swipe events

These will only trigger the gesture if it's been created with
the same GtkGesture::n-points than n_fingers in the event.

gtkgestureswipe: Handle touchpad swipe events

These will only trigger the gesture if it's been created with
the same GtkGesture::n-points than n_fingers in the event.

gtkgesturerotate: Handle touchpad pinch events

We let these through in GtkEventController::filter, and handle
these especially on GtkGesture::update.

gtkgesturezoom: Handle touchpad pinch events

We let these through in GtkEventController::filter, and handle
these especially on GtkGesture::update.

gtkgesture: Add docs blurb about touchpad gestures

Not too long though, the precautions to take are minimal.

gtkgesture: Accumulate touchpad events' dx/dy in point data

This will make the gesture "center" the pointer position, plus
the accumulated dx/dy throughout the gesture progress.

gtkgesture: Handle touchpad events

These will be mutually exclusive with touch events, so it won't
be possible to trigger gestures through mixed input and whatnot.

The accounting of touchpad events is slightly different, there
will be a single internal PointData struct, stored in the hashtable
with the NULL event sequence/key (same than pointer events in
this regard), just that the events stored will be GdkEventTouchpad*,
so will hold information about all fingers at once.

But this difference is just internal, the GtkGesture API doesn't
make explicit assumptions about the number of points (the closest
to a per-point query API is gtk_gesture_get_sequences()). All
signals emitted just contain the last changed GdkEventSequence,
and API takes GdkEventSequences, so everything is consistent with
sequence=NULL for touchpad events.

gtkgesture: Refactor n-points querying into a single function

Along the code, we're basically asking for 1) the total count of
touchpoints, and 2) the number of active touchpoints (not denied
nor ended).

Wrap both usecases into a _gtk_gesture_get_n_physical_touchpoints(),
and replace all occurrences.

gtkgesture: Filter out touchpad events by default.

The gestures that don't want touchpad gesture events are majority,
even those that want such events will only listen to subsets (eg.
pinch, swipe,...).

So it makes sense to ignore touchpad events by default, and let
subclasses opt those in.

gtkgesture: Refactor gtk_gesture_handle_event()

We'll be dealing with event types and touchpad gesture phases,
replace the switch by a battery of if/elses. Otherwise, it's
just an style change.

eventcontroller: Add private ::filter method

This will be used right before handle_event() in order to filter
out events, useful to make the previous "no touchpad events" behavior
the default, and have gesture subclasses include manually the touchpad
events they handle.

gtkwidget: Ensure touchpad events trigger the bubbling phase

For all other events, we run the bubble phase deep in the specific
::motion/button-press/release/touch handlers.

For touchpad events, it doesn't make sense to use GtkWidgetClass
slots if the intended way to deal with these are gestures, so we
run the bubble phase directly from gtk_widget_event_internal().

gtkmain: Handle rewriting of event fields during grabs for touchpad events

The window/coordinates will need translation in the case of touchpad gesture
events, same than with other pointer events.

gdk: Proxy touchpad events through the client-side window hierarchy

This includes poking the deepmost window, checking for the
GDK_TOUCHPAD_GESTURE_MASK bit set, and translating coordinates in events
accordingly.

gdk: Add GDK_TOUCHPAD_GESTURE_MASK to GdkEventMask

Users will need to manually select on this in order to receive gesture
events.

gdk: Add touchpad gesture events and event types.

Each gesture type has its separate GdkEvent struct, and begin/update/
end/cancel event types.

There is support for multi-finger swipe (3-4 fingers), and 2-finger
rotate/pinch gestures.

cellrendererpixbuf: do not clear the icon helper twice

Do not call _gtk_icon_helper_clear explicitely when the properties
are set, since the corresponding _gtk_icon_helper_set_* method
already calls clear internally.
While at it rename the reset function to make it clear that it
is calling notify for the previous image type and avoid the
notification if the image type is not changing.

color chooser: Reset show-editor on unmap

We have a testcolorchooser test, with a --edit option. It was
supposed to make the color chooser come up in edit mode, but
it didn't work ever since we dropped the ::response handler.
Fix it by resetting show-editor on unmap, instead of on map.
This way, users can set show-editor before showing the dialog,
and it will take effect.

listbox: Describe behavior change in GtkListBoxCreateWidgetFunc

Help applicaiton authors who may already be binding GtkListBox to a
GListModel by describing the change for 3.18.

https://bugzilla.gnome.org/show_bug.cgi?id=753392

Updated Spanish translation

Updated Spanish translation

label: Flesh out pango attribute support

A few attribute types were missing from the support:
rise, letterspacing, fallback, font features. Add support
for these.

Updated Norwegian bokmål translation.

Updated Spanish translation

widget-factory: Add a lock button

font chooser: Make repeated Escape cancel the dialog

The first Escape clears the search entry, the second Escape
closes the dialog.

font chooser: Make the entry activate the default

font chooser: Use browse as selection mode

font chooser: Make Escape stop search

placesview: show Disconnect instead of Eject

Since we're dealing with networks, terms like "Eject" or
the eject button are misleading, since we're not actually
ejecting but disconnecting.

Fix that by showing the appropriate icon and tooltip.

placesview: sort by path

As pointed out by Cosimo, we should sort the
items according to their path.

placesview: plug a leak

placesview: no url for networks

We are not showing the URL of network locations
anymore, since they are distracting and not
necessary.

The code, however, forgot to cleanup the URL,
so we are still showing the URL for network
locations.

Fix that by properly cleanup the URL for network
locations.

Adwaita: Don't leak underlines

The .button:link .label selector matches any label "inside" a
link button. And a label inside the context menu counts as inside
for this purpose. This causes the text-decoration property to
leak into the context menu, even though the property is not
inherited. Avoid this by tightening the selector to
.button:link > .label.

https://bugzilla.gnome.org/show_bug.cgi?id=753451

placesview: jumps to the default location

When we connect to a server, the default and expected
behavior is going to the default location, which usually
is the home directory or a writable directory.

GtkPlacesSidebar behaves properly, while GtkPlacesView
doesn't.

Fix that by jumping to the default locations instead of
the root location.

Updated Spanish translation

Remove some unused data

Remove an unused function

Remove a self-assignment

clang doesn't like those.

Fix a doc comment typo

Updated Turkish translation

Updated Portuguese translation

Formatting fixes

HighContrast: Fix undershoot in sidebars

This was just not implemented, and the sidebars solid white
background color was inherited by undershoot rectangle.

https://bugzilla.gnome.org/show_bug.cgi?id=753391

listbox: Avoid using show_all on rows

Don't use gtk_widget_show_all() on row widgets because that would
unconditionally show all of its children. This might be unwanted in case
the row implementation wants to keep some of its children hidden.

This commit changes it to use show() instead of show_all() and relies on
the row widget to control the visibility of its children itself as
appropriate.

https://bugzilla.gnome.org/show_bug.cgi?id=753392

Updated Lithuanian translation

Update Catalan translation

quartz: Implement CSD drag resize for all edges/corners

Manual resizing for CSD windows only worked with the bottom
right corner. This extends it to work for all corners and edges.

https://bugzilla.gnome.org/show_bug.cgi?id=753329

HighContrast: Improve GtkCalendar drawing

Make days gray that are not in the current month.

https://bugzilla.gnome.org/show_bug.cgi?id=753357

file chooser button: Fix some refcounting confusion

GtkFileSystem has a complicated way to handle cancellables.
You keep the cancellable pointer that is returned by
_gtk_file_system_get_info and similar methods so that you can
cancel the operation, but you do not own a reference to it.
The only place where it is ok to unref a cancellable is in
your callback, which gets handed a cancellable that you need
to unref at the end. You are expected to compare it to the
pointer you stashed away to find out if the operation has
already been superseded by a newer call, in which case you
disregard the results.

GtkFileChooserButton was following these rules for most of
the cancellables it keeps around, but it was sometimes unreffing
the cancellables that are stored in the model, which could lead
to refcount confusion and crashes. This commit makes it follow
the rules for that case too, which fixes the crash in the bug
below, and does not show up any leaks in valgrind under light
testing.

https://bugzilla.gnome.org/show_bug.cgi?id=737804

gtk/gtkcsstypesprivate.h: Fix Build

The recent change to the enum declaration for GtkCssChange actually
relied on compiler-dependent behavior, which also breaks the build on
some non-GCC compilers, such as Visual Studio.  As noted in the
G_STATIC_ASSERT line just beneath this declaration, we need to change
this enum declaration to #define's, in order to fix the build in such
situations.

https://bugzilla.gnome.org/show_bug.cgi?id=752814

3.17.6

Fix a doc comment

Documentation fixes

Updates

placesview: Fix wrong property type when creating a GtkLabel

The type of xalign property of GtkLabel is float, not int. Using wrong type
crashes GtkFileChooser on x86_64 when compiling with clang.

https://bugzilla.gnome.org/show_bug.cgi?id=753284

GtkScrolledWindow: Don't handle key event when can't scroll

Don't return that a key event was handled when the corresponding
scrollbar can not scroll.

https://bugzilla.gnome.org/show_bug.cgi?id=753256

Adwaita: calendar button fix

- provide insensitive button state for GtkCalendar buttons

https://bugzilla.gnome.org/show_bug.cgi?id=753230

color chooser: Use a popover for the context menu

It works just as well here as it does in the file chooser, and
this lets us unify the right-click and long-press behavior a bit.
We used to switch directly to the editor on long-press, now we
can show the popover, just as we do on right-click.

wayland: Don't assume drag context has a source window when finalizing

Only a drag context which was created with 'drag_begin' will be
guaranteed to have a source window at all times. Thus, in finalize we
cannot assume we can retrieve a GdkDisplay from the source_window
pointer since it may be NULL. Though, the display is only needed for
contexts created via 'drag_begin' thus we can retrieve it after
checking that is the case.

https://bugzilla.gnome.org/show_bug.cgi?id=749339